Skip to content

Россомахина Арина, М3337#42

Open
airosso wants to merge 6 commits into
itmo2019:masterfrom
airosso:master
Open

Россомахина Арина, М3337#42
airosso wants to merge 6 commits into
itmo2019:masterfrom
airosso:master

Conversation

@airosso

@airosso airosso commented May 3, 2019

Copy link
Copy Markdown

Удалить - нажать на чекбокс и кнопку "Удалить", новое письмо - "Новое письмо"

@mokhov mokhov self-assigned this May 15, 2019
@mokhov

mokhov commented May 16, 2019

Copy link
Copy Markdown
Contributor

Клик по любой области письма должен его открывать а не только по отправителю

@mokhov mokhov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Инпут не похож. Жду в субботу на очном показе

Comment thread src/index.html
<div class="main">
<div class="finder">
<input class="finder__input" placeholder="Поиск" type="text">
<label class="finder__button-close">&times;</label>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не очень понимаю здесь семантики тега <label>

Comment thread blocks/finder__input.css
@@ -0,0 +1,14 @@
.finder__input {
position: relative;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем нужно это свойство?

@airosso

airosso commented May 16, 2019

Copy link
Copy Markdown
Author

Почему очный показ?

@mokhov

mokhov commented May 18, 2019

Copy link
Copy Markdown
Contributor

Потому что задание не готово, а сейчас мы принимаем только очно

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants